Skip to content

Do not return {:ok, _} on maps with empty lines in map_put_static_value#15597

Merged
josevalim merged 3 commits into
elixir-lang:mainfrom
lukaszsamson:ls-map-put-result-empty
Jul 25, 2026
Merged

Do not return {:ok, _} on maps with empty lines in map_put_static_value#15597
josevalim merged 3 commits into
elixir-lang:mainfrom
lukaszsamson:ls-map-put-result-empty

Conversation

@lukaszsamson

@lukaszsamson lukaszsamson commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Addresses Repro 1 from #15594

Related to #15593

Assisted-By: Claude Opus 4.8

@gldubc gldubc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with adding the condition.

Comment thread lib/elixir/test/elixir/module/types/descr_test.exs Outdated
Comment thread lib/elixir/test/elixir/module/types/descr_test.exs Outdated
@josevalim
josevalim merged commit 9e78534 into elixir-lang:main Jul 25, 2026
@josevalim

Copy link
Copy Markdown
Member

💚 💙 💜 💛 ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants